Skip to content

Adding tag, gray900 #571

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
May 8, 2025
Merged

Adding tag, gray900 #571

merged 4 commits into from
May 8, 2025

Conversation

emilmilanov
Copy link
Contributor

@emilmilanov emilmilanov commented May 5, 2025

image

Copy link

changeset-bot bot commented May 5, 2025

🦋 Changeset detected

Latest commit: 42841e2

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@cypress-design/constants-tag Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

Copy link

vercel bot commented May 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
cypress-design ✅ Ready (Inspect) Visit Preview May 8, 2025 2:29pm

Copy link

cypress bot commented May 5, 2025

cypress-design    Run #2856

Run Properties:  status check passed Passed #2856  •  git commit 4bb26e2629 ℹ️: Merge 42841e2d96d9caec41046bee5350c3368cd5ea82 into cdd0897e61cae57257ca4e49e732...
Project cypress-design
Branch Review tag-gray-900
Run status status check passed Passed #2856
Run duration 05m 29s
Commit git commit 4bb26e2629 ℹ️: Merge 42841e2d96d9caec41046bee5350c3368cd5ea82 into cdd0897e61cae57257ca4e49e732...
Committer Emil Milanov
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 0
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 236
View all changes introduced in this branch ↗︎
UI Coverage  7.65%
  Untested elements 298  
  Tested elements 26  
Accessibility  99.43%
  Failed rules  0 critical   2 serious   0 moderate   1 minor
  Failed elements 23  

@@ -17,6 +17,10 @@ export const CssColor = {
default: 'bg-gray-50 text-gray-700 border-gray-100',
dark: 'bg-gray-700 text-white border-gray-700',
},
gray900: {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@emilmilanov I would recommend calling this one gray-dark maybe just to align closer to how we call our Button components.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree.

@emilmilanov
Copy link
Contributor Author

PR is ready for a review @ryanjwilke

I updated all tag variants to support outline.

The only issue I see is that white dark and gray light variants look the same. This issue existed before and is consequence of the decision that each color variant has light and dark option.

Copy link
Contributor

@ryanjwilke ryanjwilke left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems fine for now. I don't have any major concerns on my end.

@emilmilanov emilmilanov merged commit ac915e5 into main May 8, 2025
33 of 35 checks passed
@emilmilanov emilmilanov deleted the tag-gray-900 branch May 8, 2025 16:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants